Skip to content

[config reload] Modify has_timer to delayed based on recent config reload enhancements#8048

Closed
dgsudharsan wants to merge 1 commit intosonic-net:masterfrom
dgsudharsan:config_rel_cleanup
Closed

[config reload] Modify has_timer to delayed based on recent config reload enhancements#8048
dgsudharsan wants to merge 1 commit intosonic-net:masterfrom
dgsudharsan:config_rel_cleanup

Conversation

@dgsudharsan
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Clean up configuration files based on config reload enhancement sonic-net/SONiC#1203
Changed the field "has_timer" to "delayed"

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

…load enhancements

Change-Id: Ia2f627a70f95c7d67c2bb08af4408f923a323325
@dgsudharsan dgsudharsan requested a review from qiluo-msft April 14, 2023 23:01
@dgsudharsan
Copy link
Copy Markdown
Contributor Author

Since there is no separate branch for 202211, merging this PR will result in regression failure in 202211. The code changes in SONiC already take care of handling has_timer using db_migrator. As long as VERSION field in the configs is correct there should be no issues. Closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants